Skip to content

chore: add name and email prefix for myself - #150

Merged
dsp-ant merged 1 commit into
modelcontextprotocol:mainfrom
Kludex:add-marcelo-email
Jul 24, 2026
Merged

chore: add name and email prefix for myself#150
dsp-ant merged 1 commit into
modelcontextprotocol:mainfrom
Kludex:add-marcelo-email

Conversation

@Kludex

@Kludex Kludex commented Jul 24, 2026

Copy link
Copy Markdown
Member

Adds my name and email prefix to get a marcelo@modelcontextprotocol.io account, same as #148.

AI Disclaimer

This PR was developed with the assistance of either Claude or Codex. I've reviewed and verified the changes.

@Kludex
Kludex requested a review from a team as a code owner July 24, 2026 09:37
@dsp-ant
dsp-ant merged commit c129364 into modelcontextprotocol:main Jul 24, 2026
2 checks passed
dsp-ant added a commit that referenced this pull request Jul 27, 2026
GWS user accounts were only created for members whose direct roles had
google.provisionUser (lead/core maintainers, moderators, maintainers,
registry-wg). SDK maintainers and working-group members who opted in by
adding firstName/lastName/googleEmailPrefix (e.g. #148, #150, #152)
silently got no account.

- Move provisionUser from GoogleConfig to the Role level, and add it to
  the implicit-maintainer containers (sdk-maintainers, working-groups)
  plus the standalone maintainer roles (docs, inspector, mcpb,
  reference-servers, use-mcp, server-identity-wg, wg/ig facilitators).
- Eligibility now walks github.parent chains (hasProvisionUserRole),
  matching GitHub team-nesting semantics, so e.g. python-sdk members
  qualify via sdk-maintainers.
- validate-config now errors when a googleEmailPrefix would be a silent
  no-op (no eligible role and not an existing GWS user), so opt-in PRs
  like #150 fail CI instead of quietly doing nothing.

Newly provisioned on next apply: marcelo@, alexhancock@, liad@.
ochafik is now eligible but already has an account (existingGWSUser).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants